Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

documentation added on FLINT Performance. #102

Closed

Conversation

SanjaySinghRajpoot
Copy link
Contributor

@SanjaySinghRajpoot SanjaySinghRajpoot commented Sep 15, 2021

Pull Request Template

Description

documentation added on FLINT Performance.

Fixes #77

Type of change

  • This change requires a documentation update

@SanjaySinghRajpoot SanjaySinghRajpoot changed the title document on FLINT Performance documentation added on FLINT Performance. Sep 15, 2021
Copy link
Member

@HarshCasper HarshCasper left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This documentation has also not been referenced anywhere and thus would not be rendered on the original website. Please fix the same.

Do sign off the DCO as well (Instructions are provided on the checks).

docs/DeveloperWorkflow/flint_performance.rst Outdated Show resolved Hide resolved
docs/DeveloperWorkflow/flint_performance.rst Outdated Show resolved Hide resolved
accounts. Most countries are currently using systems that are not spatially explicit
like FLINT. In these systems, land cover types are aggregated and the specific achievements
of a project area are reduced to the average emission applied to that land cover type or
they cannot be identified at all within the national accounts.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this documentation is incomplete 👀

@aornugent Can we please have your inputs over the same?

Signed-off-by: SanjaySinghRajpoot <[email protected]>
Signed-off-by: SanjaySinghRajpoot <[email protected]>
@SanjaySinghRajpoot
Copy link
Contributor Author

@HarshCasper

Thank You

@@ -2,3 +2,4 @@

FLINT Performance
#################

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why do I see this as the only change?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

something went wrong. I have created a branch for another PR I made some changes in that particular branch but due to some reason, they are reflected here also. Let me close this PR make a new one with valid changes and properly signed commits.

Thank You

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Provide documentation on FLINT Performance
2 participants